Skip to content

Empty states for image reading#346

Open
rivalee wants to merge 2 commits into
mainfrom
ir-empty-states
Open

Empty states for image reading#346
rivalee wants to merge 2 commits into
mainfrom
ir-empty-states

Conversation

@rivalee

@rivalee rivalee commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

Ticket: https://nhsd-jira.digital.nhs.uk/browse/DTOSS-13234

Context

Most of our designs assume there's a large backlog of cases. What happens when there aren't enough, or aren't any?

We need some empty states for image reading.

Covering main situations:

  1. No eligible cases to read (eg you've done all the first or second reads possible)
  2. No cases at all to read, backlog empty
  3. Low number of cases in backlog, and another reader has read cases in your allotted 25, so there's an unexpectedly lower number of cases to read

Update generic start page content

To accommodate default number of cases being under 25 (more likely for the tests)

Screenshot 2026-06-25 at 16 02 14

Empty state start page for image reading (no cases)

Screenshot 2026-06-25 at 15 49 23

Interstitial / interruption when there's no more cases to read

Screenshot 2026-06-25 at 15 53 55

Session overview

  • Only 10 cases left in backlog (completed)
  • When no more cases to read, don't show new session button
localhost_3000_reading_session_fs9g26tz_your-reads

@edwardhorsford edwardhorsford temporarily deployed to manage-breas-ir-empty-s-ch0vwi June 25, 2026 14:57 Inactive
@rivalee rivalee marked this pull request as ready for review June 25, 2026 14:58
@rivalee rivalee requested review from dannychadburn and edwardhorsford and removed request for dannychadburn June 25, 2026 15:03
@dannychadburn

dannychadburn commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator

@rivalee - need to look at the 'Opinion summary' on the session overview page which shows 'X remaining' and 'X more in this session' when a session has been ended prematurely because it ran out of cases

image


{% elif userReadableTotal == 0 %}

<h2>There are no cases to read</h2>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<h2>There are no cases to read</h2>
<h2>>There are no more cases available</h2>

@dannychadburn dannychadburn Jun 26, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

avoiding duplicating 'read' and 'read' in heading and text

{% elif userReadableTotal == 0 %}

<h2>There are no cases to read</h2>
<p>All cases have been read.</p>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<p>All cases have been read.</p>
<p>You have completed all the cases you are eligible to read.</p>


<h2>Start image reading</h2>
<p>Give your opinion on the next {{ defaultSessionSize }} cases due for reading.</p>
<p>Give your opinion on cases due for reading (up to {{ defaultSessionSize }}).</p>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<p>Give your opinion on cases due for reading (up to {{ defaultSessionSize }}).</p>
<p>Give your opinion on cases due for reading.</p>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removing mentions of any case numbers in this sentence as it doesn't flow well - suggest secondary text below the button to clarify what they session size is

text: "Start now",
href: "/reading/create-session?type=all_reads"
}) }}

@dannychadburn dannychadburn Jun 26, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<br>
<span class="nhsuk-u-secondary-text-colour nhsuk-body-s">Your reading session will include up to {{ defaultSessionSize }} cases</span>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added this as secondary text below the start button as the 'up to' muddles the sentence above - it's not particularly essential info

<span class="nhsuk-caption-l">Image reading</span>
<h1 class="nhsuk-heading-l">{{ pageHeading }}</h1>

<p>All eligible cases in this session have been read.</p>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<p>All eligible cases in this session have been read.</p>
<p>You have given an opinion on all available cases.</p>

text: "Review session outcomes",
href: "/reading/session/" + sessionId
}) }}
<a href="/reading" class="nhsuk-link nhsuk-button-group__item">Exit reading</a>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<a href="/reading" class="nhsuk-link nhsuk-button-group__item">Exit reading</a>
<a href="/reading" class="nhsuk-link nhsuk-button-group__item">Exit session</a>

{% set hideStatusBar = true %}
{% set hideBackLink = true %}

{% set pageHeading = "No more cases to read" %}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
{% set pageHeading = "No more cases to read" %}
{% set pageHeading = "Session complete" %}

<div class="nhsuk-grid-column-two-thirds">
<h1>{{ pageHeading }}</h1>
<p>{{ backlogTotal }} cases require reading</p>
<p>{{ userReadableTotal }} {{ "case" | pluralise(userReadableTotal) }} available to read</p>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<p>{{ userReadableTotal }} {{ "case" | pluralise(userReadableTotal) }} available to read</p>
<p class="nhsuk-u-secondary-text-colour nhsuk-body-l">{{ userReadableTotal }} {{ "case" | pluralise(userReadableTotal) }} are awaiting an outcome</p>

@dannychadburn dannychadburn Jun 26, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Related issue https://nhsd-jira.digital.nhs.uk/browse/DTOSS-13222

This distinguishes all reads from those available to the user to read - ie, "awaiting an outcome" could cover 1st or 2nd reads, in arbitration, awaiting priors, deferred, etc

Also, have suggested some visual distinction for this paragraph - lots of similar looking lines of text on the page currently

@@ -117,13 +125,23 @@
href: "/reading/session/" + inProgressSession.id + "/resume"
}) }}

@dannychadburn dannychadburn Jun 26, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
}) }}
}) }}
<br>
<span class="nhsuk-u-secondary-text-colour app-nowrap nhsuk-body-s">{{ inProgressSession.userReadCount }} of {{ inProgressSession.targetSize }} cases read</span>

@dannychadburn dannychadburn Jun 26, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggest moving this below the start button to avoid lots of text paragraphs (similar suggestion made on the regular start page)


<h2>Resume image reading</h2>
<p>You have an image reading session in progress. Finish it before starting a new one.</p>
<p>{{ inProgressSession.userReadCount }} of {{ inProgressSession.targetSize }} cases read.</p>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removes progress count from here (added as secondary info below button to mirror suggestion on regular start page

@dannychadburn

Copy link
Copy Markdown
Collaborator

Suggest we remove the 'Start a new session' button on session overview if there are no more cases available for the user to read

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants